Sql render #1
Merged
Sql render #1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
deprecate pz_gallery deactivated menus for bringup
…tes with new implementations
… svg rendered diagrams in astro code components
bugfix wrong language inference on parent code blocks
functional headings and images
refactor: enhance language detection in code component and remove XLSX table component fix: improve error handling in dynamic page rendering
added directive completed all item types
There was a problem hiding this comment.
Pull request overview
This PR introduces a fully functional SQLite-based markdown rendering system that replaces file-based content management with a database-driven approach. The implementation enables efficient content storage, retrieval, and rendering through a structured database schema.
Key Changes:
- Implemented SQLite database integration for markdown content and assets via
structure-db.js - Added comprehensive markdown rendering pipeline with support for diagrams, tables, images, and interactive components
- Introduced server-side rendering with Astro node adapter and optional GitHub authentication
Reviewed changes
Copilot reviewed 77 out of 98 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
src/libs/structure-db.js |
Core database interface for querying documents, items, assets, and blobs from SQLite |
src/pages/index.astro, src/pages/[...url].astro |
Dynamic page rendering using database queries |
src/pages/assets/[...uid].js |
Asset serving endpoint with fallback strategies |
src/components/markdown/AstroMarkdown.astro |
Main markdown rendering component orchestrating all content types |
src/components/markdown/code/Code.astro |
Code block rendering with syntax highlighting and diagram support |
src/layout/Layout.astro, src/layout/layout_utils.js |
Page layout with navigation menus built from database |
astro.config.mjs |
Astro configuration for SSR with node adapter |
config.js |
Centralized configuration loading from manifest.yaml |
scripts/diagrams.js |
Build-time diagram generation via Kroki API |
server/server.js |
Express server for production deployment with optional auth |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| //https://developer.mozilla.org/en-US/docs/Web/SVG/Element/animate | ||
| //calcMode [discrete | linear | paced | spline] | ||
|
|
||
| function anim_radius(id,max){ |
There was a problem hiding this comment.
Unused function anim_radius.
src/layout/toc_menu_activation.js
Outdated
| } | ||
|
|
||
| function toc_menu_activation(){ | ||
| const toc_menu = document.getElementsByClassName("toc_menu")[0] |
There was a problem hiding this comment.
Unused variable toc_menu.
default on server side current page visible on menu
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fully functional Markdown sqlite db rendering